Skip to content

Add phone verification inputs - #475

Draft
oschwald wants to merge 1 commit into
mainfrom
greg/billing-phone-verification-inputs
Draft

oschwald wants to merge 1 commit into
mainfrom
greg/billing-phone-verification-inputs

Conversation

@oschwald

@oschwald oschwald commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

Adds three optional inputs to the /billing and /shipping request objects: phone_verification_method (delivered_code, network, or other), phone_was_verification_successful (boolean), and phone_verification_time (RFC 3339 date-time). They describe the most recent verification of the phone number. Both objects share one validation schema, and each value is checked.

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Add the optional phone_verification_method,
phone_was_verification_successful, and phone_verification_time inputs
to both the /billing and /shipping request objects. They describe the
most recent verification of the phone number. Both objects share the
same validation.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@oschwald
oschwald force-pushed the greg/billing-phone-verification-inputs branch from 2792b9a to a28ab6c Compare October 12, 2026 14:56
@oschwald oschwald changed the title Add billing phone verification inputs Add phone verification inputs Oct 12, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant